Repository navigation
Main2Dev @W-22462053@ Merging main to dev after v1.22.0 - #372
Conversation
RELEASE @W-22462053@ Releasing v1.22.0
|
Git2Gus App is installed but the |
aruntyagiTutu
left a comment
aruntyagiTutu
left a comment
There was a problem hiding this comment.
Content reviewed — this is the standard Main2Dev merge-back after the v1.22.0 release, aggregating already-reviewed dev commits (message sanitization #368, includeSuggestions default #369, dependency refresh #370, release #371) plus the PR-title-injection hardening fix mirrored from code-analyzer-core (passing head_ref/base_ref via env instead of ${{ }} interpolation, plus a new permissions: contents: read restriction on the workflow — good defense-in-depth addition). SHA256.md and version bump are consistent with prior releases. No code concerns.
Per the PR's own body, the title should be renamed to include 'merging' (e.g. 'Main2Dev @W-XXXXXXXX@ Merging main to dev after v1.22.0') before merge — currently still the placeholder 'Filler title. Read description and rename.'
nikhil-mittal-165
changed the title
# Conflicts: # .github/workflows/validate-pr.yml # SHA256.md
aruntyagiTutu
left a comment
aruntyagiTutu
left a comment
There was a problem hiding this comment.
Title has been renamed correctly and the PR-title-validation check now passes. Re-verified the diff after the dev→m2d merge-conflict resolution: SHA256.md and package.json correctly retain the released v1.22.0 hash/version, and the validate-pr.yml hardening (env-based head_ref/base_ref, permissions: contents: read) merged in cleanly with no regressions. CI is green. Approving.
3 participants
This branch and PR were automatically created following the successful release of v1.22.0.
It must be MERGED into dev, NOT SQUASHED OR REBASED. Squashing or rebasing this branch onto dev can cause potentially irreconcilable merge conflicts later.
As an additional safeguard and reminder, the title of this PR MUST include the word 'merging' in the description portion of the PR title, e.g., 'Main2Dev @w-xxxxxx@ Merging main to dev after vX.Y.Z'.
If there are conflicts between dev and this branch, you should do the following locally: